Skip to content

Conversation

spastorino
Copy link
Member

This is the last variant that needed to be covered for Terminator. As we've discussed with @oli-obk I've made a lot of it's fields be Strings.

r? @oli-obk

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jun 7, 2023
@rustbot
Copy link
Collaborator

rustbot commented Jun 7, 2023

This PR changes Stable MIR

cc @oli-obk, @celinval

@spastorino spastorino force-pushed the smir-terminator-3 branch 2 times, most recently from 0e3cd7a to 2431728 Compare June 7, 2023 18:05
@spastorino spastorino force-pushed the smir-terminator-3 branch from 570eeb2 to 891d6a9 Compare June 8, 2023 15:37
Copy link
Contributor

@oli-obk oli-obk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me with the comments addressed

@spastorino spastorino force-pushed the smir-terminator-3 branch 2 times, most recently from 2b711d6 to 1dfb036 Compare June 8, 2023 19:50
@spastorino spastorino force-pushed the smir-terminator-3 branch from 1dfb036 to 313143b Compare June 8, 2023 19:53
@spastorino
Copy link
Member Author

@bors r=oli-obk

@bors
Copy link
Collaborator

bors commented Jun 8, 2023

📌 Commit 313143b has been approved by oli-obk

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 8, 2023
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 9, 2023
…i-obk

Add Terminator::InlineAsm conversion from MIR to SMIR

This is the last variant that needed to be covered for Terminator. As we've discussed with `@oli-obk` I've made a lot of it's fields be `String`s.

r? `@oli-obk`
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 9, 2023
…iaskrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#112323 (Don't mention already-set fields in struct constructor missing field error)
 - rust-lang#112395 (Add Terminator::InlineAsm conversion from MIR to SMIR)
 - rust-lang#112411 (add programmerjake to portable-simd cc list)
 - rust-lang#112428 (Structurally resolve pointee in `check_pat_lit`)
 - rust-lang#112444 (Don't debug-print `Interned` or `PrivateZst`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 960d71e into rust-lang:master Jun 9, 2023
@rustbot rustbot added this to the 1.72.0 milestone Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants